Skip to content

[SPARK-58972][FOLLOWUP][DOCS] Add migration guide entry for from_protobuf wrapper unwrapping - #59245

Open
bhollis-dbx wants to merge 3 commits into
apache:masterfrom
bhollis-dbx:bhollis-dbx/spark-58972-migration-guide
Open

bhollis-dbx wants to merge 3 commits into
apache:masterfrom
bhollis-dbx:bhollis-dbx/spark-58972-migration-guide

Conversation

@bhollis-dbx

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

SPARK-58972 changed what from_protobuf returns for a present-but-empty primitive wrapper under unwrap.primitive.wrapper.types (NULL -> the scalar default). Document the behavior change in the 4.3 -> 4.4 migration guide.

Why are the changes needed?

Users should know that the behavior is changing.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Documentation only.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Opus 5.5

…obuf wrapper unwrapping

SPARK-58972 changed what from_protobuf returns for a present-but-empty
primitive wrapper under unwrap.primitive.wrapper.types (NULL -> the scalar
default). Document the behavior change in the 4.3 -> 4.4 migration guide.

Co-authored-by: Isaac <no-reply@databricks.com>
@uros-b

uros-b commented Oct 6, 2026

Copy link
Copy Markdown
Member

Thank you @bhollis-dbx for following up on this!

@uros-b

uros-b commented Oct 6, 2026

Copy link
Copy Markdown
Member

Also, thank you @kavpreetgrewal for review

@uros-b

uros-b commented Oct 6, 2026

Copy link
Copy Markdown
Member

cc @zhengruifeng, re: #58253

@uros-b
uros-b requested a review from zhengruifeng October 6, 2026 12:47
Comment thread docs/sql-migration-guide.md Outdated
Co-authored-by: Dongjoon Hyun <dongjoon@apache.org>
@bhollis-dbx

Copy link
Copy Markdown
Contributor Author

@dongjoon-hyun good call, accepted your suggestion.

@dongjoon-hyun dongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants